[fix] Server-side fetch request should have headers (#696)#2861
[fix] Server-side fetch request should have headers (#696)#2861aolose wants to merge 5 commits intosveltejs:masterfrom aolose:master
Conversation
|
| }; | ||
| } | ||
|
|
||
| //merge headers from request |
There was a problem hiding this comment.
we usually put a space after //
| //merge headers from request | |
| // merge headers from request |
There was a problem hiding this comment.
There are two spaces between words too, we should have one space only.
|
@aolose this PR would need to be rebased and my comment would need to be addressed |
aolose
changed the title
3 participants
Server-side fetch in load function will include original request headers.
I'm only test it in the Page file , if server-side fetch not in load , headers will lost :( ,
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm testand lint the project withpnpm lintandpnpm checkChangesets
pnpx changesetand following the prompts. All changesets should bepatchuntil SvelteKit 1.0